-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add test helpers to framework repo #37
Open
davidhorstmann-arm
wants to merge
81
commits into
main
Choose a base branch
from
dev/davidhorstmann-arm/add-test-helpers
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Minos Galanakis <[email protected]>
Co-authored-by: Janos Follath <[email protected]> Signed-off-by: minosgalanakis <[email protected]>
Removed discrete link referencing and updated text entries. Signed-off-by: Minos Galanakis <[email protected]>
PSA buffers and their length should follow a pattern for which the latter is named after the former as <buffer_name>_length, but this was not the case for psa_key_derivation_verify_bytes(). This makes life of crypto.h parsers (for automatic code generation) harder. This commit aims at solving this problem. Signed-off-by: Valerio Setti <[email protected]>
Signed-off-by: Thomas Daubney <[email protected]>
Signed-off-by: Thomas Daubney <[email protected]>
Signed-off-by: Thomas Daubney <[email protected]>
Also removed are all options of the form MBEDTLS_ECP_XXX_ALT as well as MBEDTLS_ECP_NO_FALLBACK. Signed-off-by: Thomas Daubney <[email protected]>
Signed-off-by: Thomas Daubney <[email protected]>
Signed-off-by: David Horstmann <[email protected]>
These now point to framework/data_files instead. Signed-off-by: David Horstmann <[email protected]>
A path containing '../../' was defined in relation to a directory in the tests/data_files directory. Since this has been moved to framework/data_files, we must update the path to refer correctly to the same location as before. Signed-off-by: David Horstmann <[email protected]>
Signed-off-by: David Horstmann <[email protected]>
Signed-off-by: David Horstmann <[email protected]>
Signed-off-by: David Horstmann <[email protected]>
Signed-off-by: David Horstmann <[email protected]>
Signed-off-by: David Horstmann <[email protected]>
Signed-off-by: David Horstmann <[email protected]>
Move the DATA_FILES_PATH_VAR variable into the generated bash rather than only variablising it in the python that generates the test script. Signed-off-by: David Horstmann <[email protected]>
Signed-off-by: David Horstmann <[email protected]>
Signed-off-by: David Horstmann <[email protected]>
…o_alt_interface Remove asymmetric crypto alt interfaces
* Removed strike-out marks and added | symbols for selection. * Added framework tracking checkbox. Signed-off-by: Minos Galanakis <[email protected]>
Signed-off-by: Ronald Cron <[email protected]>
…-to-framework Move test data files to framework
Move all the modules that constitute the crypto library as well as the associated headers to tf-psa-crypto/core for the PSA core modules and to tf-psa-crypto/drivers/builtin/src for the others. The common.h file is copied instead of being just moved as eventually they will be different in mbedtls and TF-PSA-Crypto. Some parts of it can be shared though but this will be done later, probably when adding the CMake build system in tf-psa-crypto. Signed-off-by: Ronald Cron <[email protected]>
Adjust build systems such as we can built Mbed TLS in the default and full configuration. Signed-off-by: Ronald Cron <[email protected]>
Signed-off-by: Ronald Cron <[email protected]>
Signed-off-by: Ronald Cron <[email protected]>
Signed-off-by: Ronald Cron <[email protected]>
Signed-off-by: Elena Uziunaite <[email protected]>
PR-Template: Updated the PR template with new checkboxes
Replace MBEDTLS_MD_CAN_SHA3_224 with PSA_WANT_ALG_SHA3_224
…_alt_interface Remove symmetric crypto alt interface
Signed-off-by: Elena Uziunaite <[email protected]>
Signed-off-by: Elena Uziunaite <[email protected]>
Signed-off-by: Thomas Daubney <[email protected]>
Signed-off-by: Thomas Daubney <[email protected]>
Signed-off-by: Thomas Daubney <[email protected]>
Signed-off-by: Thomas Daubney <[email protected]>
Signed-off-by: Thomas Daubney <[email protected]>
Signed-off-by: Thomas Daubney <[email protected]>
Signed-off-by: Thomas Daubney <[email protected]>
Signed-off-by: Thomas Daubney <[email protected]>
Signed-off-by: Thomas Daubney <[email protected]>
Restore previously removed option but with a more relevant name. Signed-off-by: Thomas Daubney <[email protected]>
Signed-off-by: Thomas Daubney <[email protected]>
Replace MBEDTLS_MD_CAN_SHA3_384 with PSA_WANT_ALG_SHA3_384
…erface Remove AES-like alt interface
…ify_bytes psa: fix parameters' names of psa_key_derivation_verify_bytes()
Signed-off-by: David Horstmann <[email protected]>
…mann-arm/add-test-helpers
Update paths in generating scripts that refer to tests/src. These will now generate files instead in framework/tests/src (relative to the main repo root directory). Signed-off-by: David Horstmann <[email protected]>
Update references to tests/src and tests/include in a code comment now that these live in the framework. Signed-off-by: David Horstmann <[email protected]>
davidhorstmann-arm
added
needs-review
Every commit must be reviewed by at least two team members,
needs-ci
Needs to pass CI tests
needs-reviewer
This PR needs someone to pick it up for review
priority-high
High priority - will be reviewed soon
labels
Jul 18, 2024
5 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
needs-ci
Needs to pass CI tests
needs-review
Every commit must be reviewed by at least two team members,
needs-reviewer
This PR needs someone to pick it up for review
priority-high
High priority - will be reviewed soon
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Move the contents of
tests/src
andtests/include
(excludingtests/include/alt-dummy
) to the framework and adjust references to these paths accordingly.